Skip to content

test: add relay ping rate-limit coverage + limiter LRU eviction#68

Merged
Scottcjn merged 1 commit intoScottcjn:mainfrom
sungdark:fix/relay-rate-limit-tests
Feb 26, 2026
Merged

test: add relay ping rate-limit coverage + limiter LRU eviction#68
Scottcjn merged 1 commit intoScottcjn:mainfrom
sungdark:fix/relay-rate-limit-tests

Conversation

@sungdark
Copy link
Contributor

Summary\n- add a regression test that enforces write-rate limiting on \n- add a bounded-limiter test to verify LRU eviction when is exceeded\n\n## Why\nIssue #389 requests bounded rate-limit storage and endpoint coverage for Atlas mutation paths. This PR strengthens test coverage for both the storage bound and the relay ping write path.\n\n## Notes\n- Could not execute pytest in this environment because / are unavailable in the runner.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to beacon-skill! Thanks for your first pull request.

Quick checklist:

  • Tests pass (pytest tests/)
  • New code includes SPDX license header
  • Focused on a single change

Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100)

A maintainer will review your PR soon!

@Scottcjn Scottcjn merged commit 35e7884 into Scottcjn:main Feb 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S PR: 11-50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants